-
Notifications
You must be signed in to change notification settings - Fork 33
Remove all remnants of --local #854
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #854 +/- ##
==========================================
Coverage 100.00% 100.00%
==========================================
Files 59 56 -3
Lines 3389 3173 -216
Branches 663 626 -37
==========================================
- Hits 3389 3173 -216 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
MichaelGoberling
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm
| mockRunDev.mockImplementation((config, dataDir, options, logFunc) => { | ||
| logFunc('boo') | ||
| expect(options.isLocal).toBe(undefined) | ||
| // expect(options.isLocal).toBe(undefined) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sayonara?
shazron
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Deleting code feels so good 🎶
|
Need this major version update change also: adobe/generator-app-common-lib#77 but it has complications (see PR description) |
Description
Closes #813
Related Issue
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: